Skip to content

Conversation

@Githubguy132010
Copy link
Contributor

@Githubguy132010 Githubguy132010 commented Apr 28, 2025

Context

Add the Dutch language and implement #2579

Implementation

The latest commits add extensive Dutch language support throughout the Roo-Code project. This includes:

  • Adding Dutch translations for common terms, tools, chat, and history in both the main app and the webview UI.
  • Updating relevant files such as src/i18n/locales/nl/common.json, src/i18n/locales/nl/tools.json, webview-ui/src/i18n/locales/nl/chat.json, webview-ui/src/i18n/locales/nl/common.json, and webview-ui/src/i18n/locales/nl/history.json.
  • Refactoring translation files to ensure consistency and removing redundant task statuses from the Dutch translation of the history module.
  • Updating type definitions and dependencies for improved type safety.
  • Adding Dutch translation to the README.
  • Incorporating a community-driven update to the Dutch CODE_OF_CONDUCT.md file.

No major architectural changes were made; the focus was on localization and translation accuracy, with some type updates for better maintainability. Tradeoffs included extra care to avoid breaking existing translations and ensuring keys matched across locales.

Screenshots

before after
(No Dutch available in UI) (Dutch is available as a language option; app content appears in Dutch)

How to Test

  • Switch the application language to Dutch from the settings or locale selector.
  • Navigate through the app and webview UI; all text should appear in Dutch where translations are available.
  • Specifically check sections like tools, chat, and history for Dutch translations.
  • Verify that the README and CODE_OF_CONDUCT are available in Dutch.
  • Test with various features to ensure there are no missing or broken translation keys.

Get in Touch

I'm available in the Roo Code Discord as Githubguy132010 if you have any questions or feedback about these changes.


Important

Add Dutch language support and improve type safety across the project.

  • Localization:
    • Add Dutch translations for common terms, tools, chat, and history in src/i18n/locales/nl/common.json, src/i18n/locales/nl/tools.json, and webview-ui/src/i18n/locales/nl/chat.json.
    • Update README.md, CODE_OF_CONDUCT.md, and CONTRIBUTING.md with Dutch translations.
  • Type Safety:
    • Update type definitions in webview-ui/package.json and webview-ui/src/react-app-env.d.ts.
  • UI Components:
    • Modify AutoApproveMenu.tsx and AutoApproveSettings.tsx to handle Dutch translations and improve type safety.

This description was created by Ellipsis for 787baae. You can customize this summary. It will automatically update as commits are pushed.

Githubguy132010 and others added 7 commits April 12, 2025 11:52
Add Dutch language support to the app.

* **Add Dutch translations**:
  - Add `src/i18n/locales/nl/common.json` with Dutch translations for common terms.
  - Add `src/i18n/locales/nl/tools.json` with Dutch translations for tools.
  - Add `webview-ui/src/i18n/locales/nl/chat.json` with Dutch translations for chat.
  - Add `webview-ui/src/i18n/locales/nl/common.json` with Dutch translations for common terms.
  - Add `webview-ui/src/i18n/locales/nl/history.json` with Dutch translations for history.
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
@changeset-bot
Copy link

changeset-bot bot commented Apr 28, 2025

⚠️ No Changeset found

Latest commit: 658699f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Apr 28, 2025
@ellipsis-dev
Copy link
Contributor

ellipsis-dev bot commented Apr 28, 2025

Hi @Githubguy132010,

Thank you for your contributions to the Roo-Code repository! I noticed that this pull request includes both Dutch language localization and component refactoring changes. To ensure a smoother review process, could you consider splitting the component refactoring into a separate pull request? This will help us focus on the localization changes independently and ensure each set of changes is reviewed thoroughly.

Thank you for your understanding and cooperation!

@dosubot dosubot bot added the enhancement New feature or request label Apr 28, 2025
@adamhill
Copy link
Contributor

Thanks for expanding the Roo-ser base with a new translation

Proost! 🦘❤️

<div align="center">
<sub>

Nederlands • [English](../../README.md)[Català](../ca/README.md)[Deutsch](../de/README.md)[Español](../es/README.md)[Français](../fr/README.md)[हिन्दी](../hi/README.md)[Italiano](../it/README.md)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you put this in alphabetical order, and also add the link to the other READMEs?

alwaysAllowSubtasks,
alwaysAllowExecute,
allowedCommands,
export const AutoApproveSettings: React.FC<AutoApproveSettingsProps> = ({
Copy link
Collaborator

@mrubens mrubens Apr 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mind pulling the auto-approve enhancements out into another PR? Ideally this one would be strictly translations.

@hannesrudolph hannesrudolph moved this from New to PR [Pre Approval Review] in Roo Code Roadmap Apr 28, 2025
@Githubguy132010
Copy link
Contributor Author

Closing this one as I cannot test the auto-approve UI. Also creating brand new PR for translations as requested.

@github-project-automation github-project-automation bot moved this from PR [Pre Approval Review] to Done in Roo Code Roadmap May 6, 2025
@mrubens
Copy link
Collaborator

mrubens commented May 6, 2025

Closing this one as I cannot test the auto-approve UI. Also creating brand new PR for translations as requested.

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants